-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Kakao Small Sans to Google Fonts #9015
base: main
Are you sure you want to change the base?
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Hello, I have already signed the CLA for my previous PR (#9014) where I submitted Kakao Big Sans. This PR (#9015) is part of the same project, adding Kakao Small Sans to Google Fonts. Since both submissions come from the same contributor and organization, could you please verify the CLA status for this PR as well? Thank you! |
Thank you very much for the submission. It looks very nice! I'm curious. Could you please answer the following questions:
|
Thank you for reviewing this submission. I'm very happy to contribute to Google Fonts. Kakao Small Sans (as well as Kakao Big Sans) is an original typeface, with the Korean font designed by Fontrix Inc. and the Latin font designed by Bon Min. This typeface was created as part of a project by Kakao Corp., and the copyright is fully owned by Kakao. If you have any further questions, please feel free to ask. |
I reviewed the issue after seeing the failure in the "Run ftxvalidator on new/changed fonts" check. Would it be okay for me to update the file and push the corrected version to this PR? Please let me know how to proceed. Thank you in advance. |
It looks like the "hdiutil attach font_tools.dmg" step failed due to a missing file in the QA environment. Thanks, |
Changed the font labeled 'Weight: 300' to Kakao Small Sans Light'
This step requires secrets and so is unlikely to work on forks. |
Thank you for your response. Since this seems to be a QA environment issue rather than a problem with the font files, |
Hi @on-the-heath!
@davelab6 @simoncozens Could you have a look at the proofs so we take a decision about this. Here is the complete fontbakery report + some proofs I generated: FontBakery reportfontbakery version: 0.13.1 Check results[29] KakaoSmallSans-Bold.ttf🔥 FAIL Do we have the latest version of FontBakery installed? fontbakery_version
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions. smart_dropout
🔥 FAIL Copyright notices match canonical pattern in fonts googlefonts/font_copyright
"Copyright 2020 The Familyname Project Authors (git url)" But instead we have got: "Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu" [code: bad-notice-format]
"Copyright 2020 The Familyname Project Authors (git url)" But instead we have got: "Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu" [code: bad-notice-format]
"Copyright 2020 The Familyname Project Authors (git url)" But instead we have got: "Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu" [code: bad-notice-format]🔥 FAIL Check Google Fonts glyph coverage. googlefonts/glyph_coverage
Use -F or --full-lists to disable shortening of long lists. [code: missing-codepoints]
|
KakaoSmallSans-Bold.ttf | |
---|---|
Dehinted Size | 1.3Mb |
Hinted Size | 1.3Mb |
Increase | 4 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file googlefonts/description/has_article
- ℹ️ INFO
This font doesn't have an ARTICLE.en_us.html file.
[code: missing-article]
ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.002; TTF Release', 'Version 2.002; TTF Release']
[code: version-not-detected]
[29] KakaoSmallSans-Light.ttf
🔥 FAIL Do we have the latest version of FontBakery installed? fontbakery_version
- 🔥 FAIL
Current FontBakery version is 0.13.1, while a newer 0.13.2 is already available. Please upgrade it with 'pip install -U fontbakery'
[code: outdated-fontbakery]
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions. smart_dropout
- 🔥 FAIL
The 'prep' table does not contain TrueType instructions enabling smart dropout control. To fix, export the font with autohinting enabled, or run ttfautohint on the font, or run the
gftools fix-nonhinting
script.
[code: lacks-smart-dropout]
🔥 FAIL Copyright notices match canonical pattern in fonts googlefonts/font_copyright
- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]🔥 FAIL Check Google Fonts glyph coverage. googlefonts/glyph_coverage
- 🔥 FAIL
Missing required codepoints:
- 0x0100 (LATIN CAPITAL LETTER A WITH MACRON)
- 0x0101 (LATIN SMALL LETTER A WITH MACRON)
- 0x0102 (LATIN CAPITAL LETTER A WITH BREVE)
- 0x0103 (LATIN SMALL LETTER A WITH BREVE)
- 0x0104 (LATIN CAPITAL LETTER A WITH OGONEK)
- 0x0105 (LATIN SMALL LETTER A WITH OGONEK)
- 0x0106 (LATIN CAPITAL LETTER C WITH ACUTE)
- 0x0107 (LATIN SMALL LETTER C WITH ACUTE)
- 0x010A (LATIN CAPITAL LETTER C WITH DOT ABOVE)
- 0x010B (LATIN SMALL LETTER C WITH DOT ABOVE)
- 93 more.
Use -F or --full-lists to disable shortening of long lists.
[code: missing-codepoints]⚠️ WARN Font has correct post table version? opentype/post_table_version
⚠️ WARNPost table format 3 use has niche use case problems.Please review the check rationale for additional details.
[code: post-table-version]
⚠️ WARN Does the font contain chws and vchw features? cjk_chws_feature
-
⚠️ WARNchws feature not found in font. Use chws_tool (https://github.com/googlefonts/chws_tool) to add it.
[code: missing-chws-feature] -
⚠️ WARNvchw feature not found in font. Use chws_tool (https://github.com/googlefonts/chws_tool) to add it.
[code: missing-vchw-feature]
⚠️ WARN Check if each glyph has the recommended amount of contours. contour_count
⚠️ WARNThis check inspects the glyph outlines and detects the total number of contours in each of them. The expected values are infered from the typical ammounts of contours observed in a large collection of reference font families. The divergences listed below may simply indicate a significantly different design on some of your glyphs. On the other hand, some of these may flag actual bugs in the font such as glyphs mapped to an incorrect codepoint. Please consider reviewing the design and codepoint assignment of these to make sure they are correct.
The following glyphs do not have the recommended number of contours:
- Glyph name: asterisk Contours detected: 2 Expected: 1 or 4
-
Glyph name: uni00AD Contours detected: 1 Expected: 0
-
Glyph name: germandbls Contours detected: 2 Expected: 1
-
Glyph name: napostrophe Contours detected: 1 Expected: 2
-
Glyph name: Psi Contours detected: 2 Expected: 1
-
Glyph name: musicalnote Contours detected: 2 Expected: 1
-
Glyph name: Psi Contours detected: 2 Expected: 1
-
Glyph name: asterisk Contours detected: 2 Expected: 1 or 4
-
Glyph name: germandbls Contours detected: 2 Expected: 1
-
Glyph name: musicalnote Contours detected: 2 Expected: 1
-
Glyph name: napostrophe Contours detected: 1 Expected: 2
Glyph name: uni00AD Contours detected: 1 Expected: 0
[code: contour-count]
⚠️ WARN Ensure files are not too large. file_size
⚠️ WARNFont file is 1.4Mb; ideally it should be less than 1.0Mb
[code: large-font]
⚠️ WARN Are there caret positions declared for every ligature? ligature_carets
⚠️ WARNThis font lacks caret position values for ligature glyphs on its GDEF table.
[code: lacks-caret-pos]
⚠️ WARN Check math signs have the same width. math_signs_width
⚠️ WARNThe most common width is 665 among a set of 13 math glyphs.
The following math glyphs have a different width, though:
Width = 644: multiply
Width = 920: propersubset, reflexsuperset, element, perpendicular, uni2252, similar, equivalence, reflexsubset, suchthat, angle, propersuperset
Width = 719: proportional
[code: width-outliers]⚠️ WARN Checking with ots-sanitize. ots
⚠️ WARNots-sanitize passed this file, however warnings were printed:
WARNING: maxp: Bad maxZones: 0
[code: ots-sanitize-warn]⚠️ WARN Check there are no overlapping path segments overlapping_path_segments
⚠️ WARNThe following glyphs have overlapping path segments:
* Psi (U+03A8): L<<139.0,742.0>--<139.0,741.0>> has the same coordinates as a previous segment.
[code: overlapping-path-segments]
⚠️ WARN Does the font contain a soft hyphen? soft_hyphen
⚠️ WARNThis font has a 'Soft Hyphen' character.
[code: softhyphen]
⚠️ WARN Ensure Stylistic Sets have description. stylisticset_description
-
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description] -
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description] -
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description]
⚠️ WARN Checking that the typoAscender exceeds the yMax of the /Agrave. typoascender_exceeds_Agrave
⚠️ WARNOS/2.sTypoAscender value should be greater than 991, but got 880 instead
[code: typoAscender]
⚠️ WARN Check font contains no unreachable glyphs unreachable_glyphs
⚠️ WARNThe following glyphs could not be reached by codepoint or substitution rules:
- glyph00001
- glyph00002
- glyph08825
- glyph08826
- glyph08834
- glyph08835
- glyph08853
- glyph08857
- glyph08860
- glyph08861
- 1215 more.
Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets. googlefonts/article/images
⚠️ WARNFamily metadata at . does not have an article.
[code: lacks-article]
⚠️ WARN Check for codepoints not covered by METADATA subsets. googlefonts/metadata/unreachable_subsetting
⚠️ WARNThe following codepoints supported by the font are not covered by
any subsets defined in the font's metadata file, and will never
be served. You can solve this by either manually adding additional
subset declarations to METADATA.pb, or by editing the glyphset
definitions.
- U+02C7 CARON: try adding one of: yi, tifinagh, canadian-aboriginal
- U+02D0 MODIFIER LETTER TRIANGULAR COLON: not included in any glyphset definition
- U+02D8 BREVE: try adding one of: yi, canadian-aboriginal
- U+02D9 DOT ABOVE: try adding one of: yi, canadian-aboriginal
- U+02DB OGONEK: try adding one of: yi, canadian-aboriginal
- U+02DD DOUBLE ACUTE ACCENT: not included in any glyphset definition
- U+0391 GREEK CAPITAL LETTER ALPHA: try adding one of: elbasan, math, greek
- U+0392 GREEK CAPITAL LETTER BETA: try adding one of: elbasan, math, greek
- U+0393 GREEK CAPITAL LETTER GAMMA: try adding one of: elbasan, math, greek
- U+0394 GREEK CAPITAL LETTER DELTA: try adding one of: elbasan, math, greek 639 more.
Use -F or --full-lists to disable shortening of long lists.
Or you can add the above codepoints to one of the subsets supported by the font: cyrillic
, korean
, latin
, latin-ext
⚠️ WARN Do any segments have colinear vectors? outline_colinear_vectors
⚠️ WARNThe following glyphs have colinear vectors:
* Delta (U+2206): L<<21.0,10.0>--<42.0,64.0>> -> L<<42.0,64.0>--<302.0,738.0>>
* Delta (U+2206): L<<384.0,738.0>--<644.0,64.0>> -> L<<644.0,64.0>--<665.0,10.0>>
* Euro (U+20AC): L<<149.0,402.0>--<149.0,344.0>> -> L<<149.0,344.0>--<149.0,329.0>>
* Euro (U+20AC): L<<149.0,418.0>--<149.0,402.0>> -> L<<149.0,402.0>--<149.0,344.0>>
* braceleft (U+007B): L<<20.0,327.0>--<21.0,374.0>> -> L<<21.0,374.0>--<20.0,421.0>>
* f (U+0066): L<<186.0,527.0>--<224.0,529.0>> -> L<<224.0,529.0>--<437.0,529.0>>
* glyph18920: L<<226.0,286.0>--<238.0,275.0>> -> L<<238.0,275.0>--<512.0,10.0>>
* glyph18929: L<<300.0,178.0>--<300.0,503.0>> -> L<<300.0,503.0>--<300.0,569.0>>
* glyph19093: L<<20.0,227.0>--<21.0,274.0>> -> L<<21.0,274.0>--<20.0,321.0>>
* gradient (U+2207): L<<460.0,1.0>--<242.0,378.0>> -> L<<242.0,378.0>--<25.0,755.0>>
* 42 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-colinear-vectors]⚠️ WARN Check the direction of the outermost contour in each glyph outline_direction
⚠️ WARNThe following glyphs have a counter-clockwise outer contour:
* copyright (U+00A9) has a counter-clockwise outer contour
* glyph18966 has a counter-clockwise outer contour
* glyph19030 has a counter-clockwise outer contour
* glyph19040 has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* j (U+006A) has a counter-clockwise outer contour
* j (U+006A) has a counter-clockwise outer contour
* m (U+006D) has a counter-clockwise outer contour
* 59 more.
Use -F or --full-lists to disable shortening of long lists.
[code: ccw-outer-contour]⚠️ WARN Do outlines contain any jaggy segments? outline_jaggy_segments
⚠️ WARNThe following glyphs have jaggy segments:
* a (U+0061): B<<420.0,523.0>-<476.0,481.0>-<496.0,396.0>>/L<<496.0,396.0>--<496.0,552.0>> = 13.24051991518721
* a (U+0061): L<<496.0,10.0>--<496.0,172.0>>/B<<496.0,172.0>-<476.0,84.0>-<420.0,40.5>> = 12.80426606528674
* aacute (U+00E1): B<<420.0,523.0>-<476.0,481.0>-<496.0,396.0>>/L<<496.0,396.0>--<496.0,552.0>> = 13.24051991518721
* aacute (U+00E1): L<<496.0,10.0>--<496.0,172.0>>/B<<496.0,172.0>-<476.0,84.0>-<420.0,40.5>> = 12.80426606528674
* acircumflex (U+00E2): B<<420.0,523.0>-<476.0,481.0>-<496.0,396.0>>/L<<496.0,396.0>--<496.0,552.0>> = 13.24051991518721
* acircumflex (U+00E2): L<<496.0,10.0>--<496.0,172.0>>/B<<496.0,172.0>-<476.0,84.0>-<420.0,40.5>> = 12.80426606528674
* adieresis (U+00E4): B<<420.0,523.0>-<476.0,481.0>-<496.0,396.0>>/L<<496.0,396.0>--<496.0,552.0>> = 13.24051991518721
* adieresis (U+00E4): L<<496.0,10.0>--<496.0,172.0>>/B<<496.0,172.0>-<476.0,84.0>-<420.0,40.5>> = 12.80426606528674
* agrave (U+00E0): B<<420.0,523.0>-<476.0,481.0>-<496.0,396.0>>/L<<496.0,396.0>--<496.0,552.0>> = 13.24051991518721
* agrave (U+00E0): L<<496.0,10.0>--<496.0,172.0>>/B<<496.0,172.0>-<476.0,84.0>-<420.0,40.5>> = 12.80426606528674
* 79 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-jaggy-segments]⚠️ WARN Do outlines contain any semi-vertical or semi-horizontal lines? outline_semi_vertical
⚠️ WARNThe following glyphs have semi-vertical/semi-horizontal lines:
* Beta (U+0392): L<<370.0,748.0>--<575.0,747.0>>
* Lambda (U+039B): L<<668.0,27.0>--<669.0,675.0>>
* glyph14846: L<<178.0,195.0>--<174.0,676.0>>
* glyph14846: L<<236.0,676.0>--<239.0,195.0>>
* glyph14846: L<<411.0,197.0>--<414.0,676.0>>
* glyph14846: L<<476.0,676.0>--<472.0,200.0>>
* glyph14854: L<<154.0,193.0>--<150.0,671.0>>
* glyph14854: L<<211.0,671.0>--<214.0,193.0>>
* glyph14854: L<<342.0,194.0>--<346.0,671.0>>
* glyph14873: L<<178.0,195.0>--<174.0,676.0>>
* 97 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-semi-vertical]⚠️ WARN Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x02 - Use grayscale rendering
[code: ranges]⚠️ WARNThe gasp range 0xFFFF value 0x02 should be set to 0x0F.
[code: unset-flags]
⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table. googlefonts/meta/script_lang_tags
⚠️ WARNThis font file does not have a 'meta' table.
[code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths superfamily/list
- ℹ️ INFO
.
[code: family-path]
ℹ️ INFO Show hinting filesize impact. hinting_impact
- ℹ️ INFO
Hinting filesize impact:
KakaoSmallSans-Light.ttf | |
---|---|
Dehinted Size | 1.4Mb |
Hinted Size | 1.4Mb |
Increase | 8 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file googlefonts/description/has_article
- ℹ️ INFO
This font doesn't have an ARTICLE.en_us.html file.
[code: missing-article]
ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.002; TTF Release', 'Version 2.002; TTF Release']
[code: version-not-detected]
[29] KakaoSmallSans-Regular.ttf
🔥 FAIL Do we have the latest version of FontBakery installed? fontbakery_version
- 🔥 FAIL
Current FontBakery version is 0.13.1, while a newer 0.13.2 is already available. Please upgrade it with 'pip install -U fontbakery'
[code: outdated-fontbakery]
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions. smart_dropout
- 🔥 FAIL
The 'prep' table does not contain TrueType instructions enabling smart dropout control. To fix, export the font with autohinting enabled, or run ttfautohint on the font, or run the
gftools fix-nonhinting
script.
[code: lacks-smart-dropout]
🔥 FAIL Copyright notices match canonical pattern in fonts googlefonts/font_copyright
- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]- 🔥 FAIL
Name Table entry: Copyright notices should match a pattern similar to:
"Copyright 2020 The Familyname Project Authors (git url)"
But instead we have got:
"Copyright (c) 2021 Kakao Corp. All rights reserved. Font designed by FONTRIX Inc. & Min Bon & Park Min-gyu"
[code: bad-notice-format]🔥 FAIL Check Google Fonts glyph coverage. googlefonts/glyph_coverage
- 🔥 FAIL
Missing required codepoints:
- 0x0100 (LATIN CAPITAL LETTER A WITH MACRON)
- 0x0101 (LATIN SMALL LETTER A WITH MACRON)
- 0x0102 (LATIN CAPITAL LETTER A WITH BREVE)
- 0x0103 (LATIN SMALL LETTER A WITH BREVE)
- 0x0104 (LATIN CAPITAL LETTER A WITH OGONEK)
- 0x0105 (LATIN SMALL LETTER A WITH OGONEK)
- 0x0106 (LATIN CAPITAL LETTER C WITH ACUTE)
- 0x0107 (LATIN SMALL LETTER C WITH ACUTE)
- 0x010A (LATIN CAPITAL LETTER C WITH DOT ABOVE)
- 0x010B (LATIN SMALL LETTER C WITH DOT ABOVE)
- 93 more.
Use -F or --full-lists to disable shortening of long lists.
[code: missing-codepoints]⚠️ WARN Font has correct post table version? opentype/post_table_version
⚠️ WARNPost table format 3 use has niche use case problems.Please review the check rationale for additional details.
[code: post-table-version]
⚠️ WARN Does the font contain chws and vchw features? cjk_chws_feature
-
⚠️ WARNchws feature not found in font. Use chws_tool (https://github.com/googlefonts/chws_tool) to add it.
[code: missing-chws-feature] -
⚠️ WARNvchw feature not found in font. Use chws_tool (https://github.com/googlefonts/chws_tool) to add it.
[code: missing-vchw-feature]
⚠️ WARN Check if each glyph has the recommended amount of contours. contour_count
⚠️ WARNThis check inspects the glyph outlines and detects the total number of contours in each of them. The expected values are infered from the typical ammounts of contours observed in a large collection of reference font families. The divergences listed below may simply indicate a significantly different design on some of your glyphs. On the other hand, some of these may flag actual bugs in the font such as glyphs mapped to an incorrect codepoint. Please consider reviewing the design and codepoint assignment of these to make sure they are correct.
The following glyphs do not have the recommended number of contours:
- Glyph name: asterisk Contours detected: 2 Expected: 1 or 4
-
Glyph name: uni00AD Contours detected: 1 Expected: 0
-
Glyph name: germandbls Contours detected: 2 Expected: 1
-
Glyph name: napostrophe Contours detected: 1 Expected: 2
-
Glyph name: musicalnote Contours detected: 2 Expected: 1
-
Glyph name: asterisk Contours detected: 2 Expected: 1 or 4
-
Glyph name: germandbls Contours detected: 2 Expected: 1
-
Glyph name: musicalnote Contours detected: 2 Expected: 1
-
Glyph name: napostrophe Contours detected: 1 Expected: 2
Glyph name: uni00AD Contours detected: 1 Expected: 0
[code: contour-count]
⚠️ WARN Ensure files are not too large. file_size
⚠️ WARNFont file is 1.4Mb; ideally it should be less than 1.0Mb
[code: large-font]
⚠️ WARN Are there caret positions declared for every ligature? ligature_carets
⚠️ WARNThis font lacks caret position values for ligature glyphs on its GDEF table.
[code: lacks-caret-pos]
⚠️ WARN Check math signs have the same width. math_signs_width
⚠️ WARNThe most common width is 678 among a set of 13 math glyphs.
The following math glyphs have a different width, though:
Width = 644: multiply
Width = 920: propersubset, reflexsuperset, element, perpendicular, uni2252, similar, equivalence, reflexsubset, suchthat, angle, propersuperset
Width = 710: proportional
[code: width-outliers]⚠️ WARN Checking with ots-sanitize. ots
⚠️ WARNots-sanitize passed this file, however warnings were printed:
WARNING: maxp: Bad maxZones: 0
[code: ots-sanitize-warn]⚠️ WARN Check there are no overlapping path segments overlapping_path_segments
⚠️ WARNThe following glyphs have overlapping path segments:
* glyph09228: L<<475.0,66.0>--<339.0,66.0>> has the same coordinates as a previous segment.
* glyph13737: L<<322.0,76.0>--<598.0,76.0>> has the same coordinates as a previous segment.
* uni3081 (U+3081): L<<178.0,720.0>--<178.0,718.0>> has the same coordinates as a previous segment.
* glyph18941: L<<347.0,573.0>--<347.0,574.0>> has the same coordinates as a previous segment.
[code: overlapping-path-segments]
⚠️ WARN Does the font contain a soft hyphen? soft_hyphen
⚠️ WARNThis font has a 'Soft Hyphen' character.
[code: softhyphen]
⚠️ WARN Ensure Stylistic Sets have description. stylisticset_description
-
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description] -
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description] -
⚠️ WARNThe stylistic set ss01 lacks a description string on the 'name' table.
[code: missing-description]
⚠️ WARN Checking that the typoAscender exceeds the yMax of the /Agrave. typoascender_exceeds_Agrave
⚠️ WARNOS/2.sTypoAscender value should be greater than 990, but got 880 instead
[code: typoAscender]
⚠️ WARN Check font contains no unreachable glyphs unreachable_glyphs
⚠️ WARNThe following glyphs could not be reached by codepoint or substitution rules:
- glyph00001
- glyph00002
- glyph08825
- glyph08826
- glyph08834
- glyph08835
- glyph08853
- glyph08857
- glyph08860
- glyph08861
- 1216 more.
Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets. googlefonts/article/images
⚠️ WARNFamily metadata at . does not have an article.
[code: lacks-article]
⚠️ WARN Check for codepoints not covered by METADATA subsets. googlefonts/metadata/unreachable_subsetting
⚠️ WARNThe following codepoints supported by the font are not covered by
any subsets defined in the font's metadata file, and will never
be served. You can solve this by either manually adding additional
subset declarations to METADATA.pb, or by editing the glyphset
definitions.
- U+02C7 CARON: try adding one of: yi, tifinagh, canadian-aboriginal
- U+02D0 MODIFIER LETTER TRIANGULAR COLON: not included in any glyphset definition
- U+02D8 BREVE: try adding one of: yi, canadian-aboriginal
- U+02D9 DOT ABOVE: try adding one of: yi, canadian-aboriginal
- U+02DB OGONEK: try adding one of: yi, canadian-aboriginal
- U+02DD DOUBLE ACUTE ACCENT: not included in any glyphset definition
- U+0391 GREEK CAPITAL LETTER ALPHA: try adding one of: elbasan, math, greek
- U+0392 GREEK CAPITAL LETTER BETA: try adding one of: elbasan, math, greek
- U+0393 GREEK CAPITAL LETTER GAMMA: try adding one of: elbasan, math, greek
- U+0394 GREEK CAPITAL LETTER DELTA: try adding one of: elbasan, math, greek 639 more.
Use -F or --full-lists to disable shortening of long lists.
Or you can add the above codepoints to one of the subsets supported by the font: cyrillic
, korean
, latin
, latin-ext
⚠️ WARN Do any segments have colinear vectors? outline_colinear_vectors
⚠️ WARNThe following glyphs have colinear vectors:
* Delta (U+2206): L<<26.0,10.0>--<57.0,89.0>> -> L<<57.0,89.0>--<301.0,739.0>>
* Delta (U+2206): L<<425.0,739.0>--<669.0,89.0>> -> L<<669.0,89.0>--<701.0,10.0>>
* K (U+004B): L<<313.0,384.0>--<328.0,368.0>> -> L<<328.0,368.0>--<665.0,10.0>>
* K (U+004B): L<<656.0,739.0>--<328.0,399.0>> -> L<<328.0,399.0>--<313.0,384.0>>
* braceright (U+007D): L<<380.0,439.0>--<380.0,375.0>> -> L<<380.0,375.0>--<380.0,310.0>>
* f (U+0066): L<<221.0,527.0>--<284.0,530.0>> -> L<<284.0,530.0>--<462.0,530.0>>
* glyph09228: L<<475.0,66.0>--<339.0,66.0>> -> L<<339.0,66.0>--<255.0,66.0>>
* glyph09228: L<<559.0,66.0>--<475.0,66.0>> -> L<<475.0,66.0>--<339.0,66.0>>
* glyph13737: L<<598.0,76.0>--<322.0,76.0>> -> L<<322.0,76.0>--<240.0,76.0>>
* glyph13737: L<<680.0,76.0>--<598.0,76.0>> -> L<<598.0,76.0>--<322.0,76.0>>
* 61 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-colinear-vectors]⚠️ WARN Check the direction of the outermost contour in each glyph outline_direction
⚠️ WARNThe following glyphs have a counter-clockwise outer contour:
* glyph18966 has a counter-clockwise outer contour
* glyph19030 has a counter-clockwise outer contour
* glyph19040 has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* ij (U+0133) has a counter-clockwise outer contour
* j (U+006A) has a counter-clockwise outer contour
* j (U+006A) has a counter-clockwise outer contour
* m (U+006D) has a counter-clockwise outer contour
* onehalf (U+00BD) has a counter-clockwise outer contour
* 42 more.
Use -F or --full-lists to disable shortening of long lists.
[code: ccw-outer-contour]⚠️ WARN Do outlines contain any jaggy segments? outline_jaggy_segments
⚠️ WARNThe following glyphs have jaggy segments:
* Mu (U+039C): B<<661.5,344.5>-<700.0,508.0>-<739.0,673.0>>/L<<739.0,673.0>--<428.0,16.0>> = 12.032608679583612
* a (U+0061): B<<449.0,479.0>-<467.0,448.0>-<476.0,406.0>>/L<<476.0,406.0>--<476.0,555.0>> = 12.094757077012089
* a (U+0061): L<<476.0,10.0>--<476.0,167.0>>/B<<476.0,167.0>-<467.0,124.0>-<449.0,92.0>> = 11.821488340607226
* aacute (U+00E1): B<<449.0,479.0>-<467.0,448.0>-<476.0,406.0>>/L<<476.0,406.0>--<476.0,555.0>> = 12.094757077012089
* aacute (U+00E1): L<<476.0,10.0>--<476.0,167.0>>/B<<476.0,167.0>-<467.0,124.0>-<449.0,92.0>> = 11.821488340607226
* acircumflex (U+00E2): B<<449.0,479.0>-<467.0,448.0>-<476.0,406.0>>/L<<476.0,406.0>--<476.0,555.0>> = 12.094757077012089
* acircumflex (U+00E2): L<<476.0,10.0>--<476.0,167.0>>/B<<476.0,167.0>-<467.0,124.0>-<449.0,92.0>> = 11.821488340607226
* adieresis (U+00E4): B<<449.0,479.0>-<467.0,448.0>-<476.0,406.0>>/L<<476.0,406.0>--<476.0,555.0>> = 12.094757077012089
* adieresis (U+00E4): L<<476.0,10.0>--<476.0,167.0>>/B<<476.0,167.0>-<467.0,124.0>-<449.0,92.0>> = 11.821488340607226
* agrave (U+00E0): B<<449.0,479.0>-<467.0,448.0>-<476.0,406.0>>/L<<476.0,406.0>--<476.0,555.0>> = 12.094757077012089
* 96 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-jaggy-segments]⚠️ WARN Do outlines contain any semi-vertical or semi-horizontal lines? outline_semi_vertical
⚠️ WARNThe following glyphs have semi-vertical/semi-horizontal lines:
* glyph11067: L<<778.0,818.0>--<776.0,348.0>>
* glyph11133: L<<778.0,818.0>--<776.0,348.0>>
* glyph13109: L<<544.0,368.0>--<545.0,803.0>>
* glyph14846: L<<167.0,207.0>--<164.0,660.0>>
* glyph14846: L<<247.0,660.0>--<250.0,207.0>>
* glyph14846: L<<396.0,209.0>--<399.0,660.0>>
* glyph14846: L<<482.0,660.0>--<479.0,213.0>>
* glyph14854: L<<222.0,656.0>--<225.0,205.0>>
* glyph14873: L<<167.0,207.0>--<164.0,660.0>>
* glyph14873: L<<247.0,660.0>--<250.0,207.0>>
* 93 more.
Use -F or --full-lists to disable shortening of long lists.
[code: found-semi-vertical]⚠️ WARN Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x02 - Use grayscale rendering
[code: ranges]⚠️ WARNThe gasp range 0xFFFF value 0x02 should be set to 0x0F.
[code: unset-flags]
⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table. googlefonts/meta/script_lang_tags
⚠️ WARNThis font file does not have a 'meta' table.
[code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths superfamily/list
- ℹ️ INFO
.
[code: family-path]
ℹ️ INFO Show hinting filesize impact. hinting_impact
- ℹ️ INFO
Hinting filesize impact:
KakaoSmallSans-Regular.ttf | |
---|---|
Dehinted Size | 1.4Mb |
Hinted Size | 1.4Mb |
Increase | 8 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file googlefonts/description/has_article
- ℹ️ INFO
This font doesn't have an ARTICLE.en_us.html file.
[code: missing-article]
ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.002; TTF Release', 'Version 2.002; TTF Release']
[code: version-not-detected]
[1] Family checks
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 12 | 60 | 340 | 16 | 246 | 0 |
0% | 0% | 2% | 9% | 50% | 2% | 36% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
Add sources' location
Hi @emmamarichal, Thank you for your feedback! I’m happy to see the progress on this. I truly appreciate your support. |
In terms of design, Latin and Hangul looks good to me. Things I noticed:
In the METADATA.pb,
|
This PR adds the Kakao Small Sans font with the required metadata files.